Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added: Fuzzy Search/File Map for Detecting File Updates #2479

Merged
merged 2 commits into from
Jan 20, 2025

Conversation

Sewer56
Copy link
Member

@Sewer56 Sewer56 commented Jan 15, 2025

This is a very small PR; just the stuff I added yesterday morning & afternoon.

This implements a 'fuzzy search' strategy for mapping that aren't the file_updates field in the V1 API; and updates all of the related documentation around updates,

This should be sufficient to make it work on 40-70% of mod pages (random guess), in the absence of proper links.

There are still some improvements that can be made, but for now I've decided to keep it relatively simple; i.e. covering most possible cases in minimum amount of work. Just enough to make it work on hopefully over half the mods; for a working prototype.

@Sewer56 Sewer56 requested a review from Al12rs January 15, 2025 10:19
@Sewer56 Sewer56 self-assigned this Jan 15, 2025
@halgari
Copy link
Collaborator

halgari commented Jan 15, 2025

I thought we were just going with whatever file was newer, until we got the rest of the UI done for this.

@Al12rs Al12rs removed their request for review January 16, 2025 09:48
Copy link
Contributor

@Al12rs Al12rs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of this code will have to be thrown out when we get to actually writing an accurate update check, and tests will need to be updated accordingly.
That is unfortunate, but I don't think there is a point in delaying the PR to removing it now or something as opposed to later on.

@Sewer56 Sewer56 merged commit 6a2754f into main Jan 20, 2025
12 checks passed
@Al12rs Al12rs deleted the update-individual-files branch January 20, 2025 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants